Skip to content

Fix cards#119

Open
achikin wants to merge 4 commits into
masterfrom
fix-cards
Open

Fix cards#119
achikin wants to merge 4 commits into
masterfrom
fix-cards

Conversation

@achikin

@achikin achikin commented Jul 3, 2018

Copy link
Copy Markdown
Member
  • Speaker's card layout replaced with flex
  • Speaker's cards are no equal height

screen shot 2018-07-03 at 19 54 23

  • Page title is centered on narrow screen

screen shot 2018-07-03 at 19 54 36

  • Встречи преобразованы в лист
    screen shot 2018-07-04 at 14 37 16

Comment thread speakers.html
</h5>
</div>
<div class="card speaker-card card-margin">
<a href={{speaker.url}}><img class="card-img-top" src="{{speaker.avatar}}" alt="{{speaker.title}}"/></a>

@trubachev trubachev Jul 3, 2018

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

можно всю карточку обернуть в <a>

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Согласен.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Так слетает автоматическое выравнивание по высоте в карточке.

@aikikode

aikikode commented Jul 3, 2018

Copy link
Copy Markdown
Contributor

Карточки спикеров хорошо получились, но во встречах теперь тяжело отследить их порядок. Они идут в три колонки каждая снизу вверх по возрастанию - не очень удобно. Может там также зафорсить размер изображения и привести все встречи к одному виду?

@achikin

achikin commented Jul 4, 2018

Copy link
Copy Markdown
Member Author

@aikikode я бы вообще их списком сделал.

@aikikode

aikikode commented Jul 4, 2018

Copy link
Copy Markdown
Contributor

@achikin давай зафорсим единый размер для всех картинок встреч - так эта страничка будет выглядеть так же ровно, как и страница спикеров.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants